Minimally invasive set of NEW_STRING changes for mmo.

Beter fxi in gdb

Leak fix in gdb.

Leak fixes for gdb.

Bring in NEW_STRINGS for gdb, saroute, igc, garmin, delbin, and an1.
Not because they actually work (in fact, they're all believed problematic)
but just to get them committed.

Fix various leaks introduced in the non-NEW_STRING case when refactoring
surrounding code to be more Qt-ish.

Rollback two formats where I introduced a regression in the non-NEW_STRINGS
case
svn merge -r4639:4637 stmwpp.cc mmo.cc

Misc minor fixes for the !NEW_STRINGS case.

Bring in csv_util.cc from NEW_STRING.

Commit the (hopefully) non-contentuous NEW_STRING changes.  This is the
batch that's largely mechanical and for formats, at least mostly passing
testo.  It's all still disabled by default and much of it is pretty dicey,
but this patch has been growing in my tree and I wanted to get it committed.

Later, when we take out the NEW_STRING stuff, I'll refactor the stuff around
these changes which were often done to preserve fidelity to the original C
versions even when there is a better way now.

Begin landing NEW_STRINGS work: common utilities.

Take another run at NMN4.

Pad internal buffer to appease -fsanitize=address.  format_garmin_xt_decrypt_trk_blk is definitely going one byte too deep on that final byte and this is probably going to come back to byte me.

Revert nmn4 change for now.  I'll deal with it later.

Make Navigon reader use QString more effectively.

Scratch in waypoint proximity in Garmin GPX writer.

Warning cleanpus from Conrad.

Fix leak I introdued in new "obvious" code yesterday.  Brace ourselves for
more of these soon...

Update .pro file from Gerhard.

Bring in some of the straight-forward pieces from NEW_STRINGS work.

A completely mechanical search and replace to swap the first argument
for an xml_generic callback with a typdef that can be swapped at compile
time.

Yes, this is a hint of things to come to reduce future bulky merges.

Compilation fixes.

Land QString changes that don't require (blatant) conditional compilation
that are really mechanical.

Make unicsv a little more QString ready internally.

Bring forward uncontested change from NEW_STRINGS work to fix non-NEW_STRINGS build break.

Commit the least contestable part of a CL I have in progress. Abandon the idea of a stepping stone "Strings" class.  Just go straight to QString, currently behind a #define.

Tweak testo to use more common stdout capture.

Guilhem Bonnefile tweaks (old) Magellan Explorist PMGNCMD,END issues.

Warning fixes from Conrad.

add utf-8 bom check and well-formed xml check to testo.

Warning cleanups from Conrad.

change illegal xml character stripping from QRegExp/QRegularExpression to QTextCodec for efficiency.  Thanks Conrad.

Preserve note for future possibilities of strip_html.

More pulling at removing C strings internally: unicsv_print_str and strenquote.

A victim of this is the seemingly unused sportsim format. It was doing
stupid things in the format definition with high byte characters and
unclear encodings.  After a few hours trying to save it, I just chopped it.

Replace guts of manual trimmed_strdup in tef reader.  Reduce type thrash.

Remove open-coded Vector in unicsv.  Make better use of QString internally.

Commit 'ptesto', a script for running valgrinds in parallel.  Really crude,
but still kind of handy.

Correct leak in bcr that I just introduced when I partially rolled back a failed experiment in making S1 and S2 into QStrings.

A bulky, but rather brainless pass at treating the important strings in
struct waypoint as a class instead of C string.  Of course, that struct
has one member - a C string - but this starts to hightlight the places
that are going to be problem areas for us.

With NEW_STRINGS turned on, we basically run, but suffer from a double
encoding of non-ascii strings.  Every format seems to be affected, so
it's probably in CET somewhere.  This isn't a QString thing in any obvious
way, as the underlying data types are still actually C Strings.

From here, additional passes can be made, either turning the individual
string pointer conversions off and whacking on a few files at a time or
going full money and making String a QString.

It does highlight the need for better vararg-style functions like warning,
fatal, and our debugging printfs to to know about stream operators.

Fixes for optionsdlg: only use ranges when we HAVE ranges (and they're not equal).  Let help link directly to the correct page.
Whitespace cleanup.

catch a straggler up with qt include renaming.

add xmlstreamwriter methods analogous to QXmlStreamWriter that strip illegal characters.

Get the gui ready for Qt5 while maintaining support for Qt4.

catch gui translation files up with recent source renaming and UpgradeCheck class work.

Minor internal changes to make new_strings land better.

Fix mem leak I introduced earlier tonight (this morning?) in ozi.

Pull out a little more char* from our core infrastructure.  Add shims to turn down case_ignore_strmp and case_ignore_strncmp().
Tweak reference files to deal with more correct(?) ordering in edge cases.
Start sketching a shim layer for shortname, description, and text, but increasingly I'm of the opinion that our only options are either do it all in one submit (eeek!) or to add additional members of struct waypt that are allocated/copied at waypt_add time so we can then at least separate readers and writers.  Instinct tells me the latter will result in harsh punishment as we'll have members that are out of sync and it will be difficult to flush them out.  So substantial parts of this CL may get tossed.

Steve replaces use of deprecated QHttp with QNetworkAccessManager.

Peter Popovec tweaks gtrnctr to fix sometimes missing lap info on write.

Remove explicit xstrdup(CSTR... in favor of helper.

Style Cleanup from last commit.

Dirk Stitchling adds support for Navin/ZNEX miniHomer 2.8 with 10Hz time.

Fix (probably) memory leak in xmltag converion.

Move remaining char*'s in xmltag to QStrings.  Tweak various callers and
copy ctors.   vtesto on CentOS 6.4 seems no worse for the wear.

Remove some read-only struct members in xml_tag.  Passes vtesto.

Fix another new/xfree mismatch.

Correct allocation in xml_tag pseudo copy constructor.

A very conservative move of xml_tag into a class of its own with
headers of its own. Tweak xmalloc/xfree to new/delete.   Regenerate deps.

fix waypoint counting errors in tpo and igc readers.
add a filter to check the counts against the lists.

fix memory leaks in cup, easygps, mapsend, osm, pcx and skyforce exposed by test-all, valgrind and patience.

fix issue with naviguide reader double counting route waypts.

fix bug in garmin_txt reader that could result in uninitialised value(s) being used when the input does not contain "Date Modified" data.

fix iblue styles, writing was causing usage of uninitialised byte(s).
fix iblue style prologue to match written fields, text taken from 2010-09-13 email.
modify mkstyle.sh to produce code compatible with our astylerc options.

improve support for testo running from alternate directories.

add option to capture valgrind output in a log file.

fix some url related memory leaks.
add constructor for UrlLink from char*.

Add valgrind leak suppressiosn for Fedora 18 to improve signal:noise.
Inspired by http://wiki.wxwidgets.org/Valgrind_Suppression_File_Howto
I'm not terribly confident these are as tight as can be (and I fear they
may be too loose in come cases) but it really helps "interesting" issues
pop out over system library loss.

Remove unnecessary strdup in garmin_gpi.

Fix obscure leak in GPX reader that I introduced.  Combo.gpx from humminbird
now doesn't lose 80K.
Tested:
  Manual inspection of valgrind --show-leaks on that specific file.
  Full valgrind on Fedora.

report errors detected by QXmlStreamReader in the geo reader.
avoid infinite loop with malformed input in the geo reader.
avoid segmentation faults with malformed input in the geo reader.

correct comment about readElementText behavior.

add error checking QXmlStreamReader errors in xmlgeneric.

Improve coment re:setCodecForCStrings.

Gerhard brings MSVC up to speed.
Resync GPSBabel.pro
Kill mkwintesto
Various MSVC weirdness.
A change in main to set character encoding that really doesn't make sense
to me since he changed only the <Qt5 case and he's on Qt5.

Another fix in gpx.cc.  With this, vtesto runs to completion on Ubuntu 12.04.

Fix use-after-read that I just introduced in gpx.cc

Replace a little-used Date field in unicsv with QDateTime for little reason
other than I was looking at it for the test failure and it was unnecessarily
unsightly.

Lock unicsv test to UTC.

Actually commit leak fix I just told Steve I committed before lunch.  {blush}

add error detection for navicache reader.

Another cleanup pass on GPX reader.  Eliminate vmem and several redundant
QString->C string roundtrips, making them explicit where needed.

Actually use autoheader to generate config.h.in instead of manual a manual edit.
It looks like this hasn't been done for a while.

rip expat out of the build system.
note that msvc/GPSBabel.vcxproj still needs to be updated to do this.

gpx reader based on QXmlStreamReader goes live.

Rename GUI *.cpp to *.cc to match convention in core.

checkpoint new gpx reader based on QXmlStreamReader instead of expat.
the new gpx reader is disabled.

eliminate weird input mode of gpx reader.
add a test to make sure standard input works.

Florian Unger adds rptname, rptdigit options to tranform filter.

How many times do I have to kill axim and psp?  Die, die, die!

Delete some dead code.

Drop dead uuid code.

Add geocaching coverage to unicsv reader/writer test.

Minor warning cleanups for Fedora.

Add a KML read test, lame as it is...

Whittle away a few more C-style date handlers.

fix another link related memory issue with the gpx reader.

fix a bug in the gpx reader that could cause the character data from the wrong element to become associated with the link/text element.

Kludge around probably TZ issue that I introduced in hiketech.

Convert remaining users (glogbook, hiketech) of xmlgeneric
writer to XMLStreamwriter. Drop a whole lot of (kinda dumb) code.

correct error in computing time spans related to DST.
add toMSecsSinceEpoch() for Qt versions before 4.7.